scrolledwindow: Fix typo in get_preferred_height calculation
authorDebarshi Ray <debarshir@gnome.org>
Mon, 16 May 2016 19:13:14 +0000 (21:13 +0200)
committerDebarshi Ray <debarshir@gnome.org>
Tue, 17 May 2016 05:39:24 +0000 (07:39 +0200)
When we are beginning to calculate the height, if the vscrollbar_policy
is not GTK_POLICY_NEVER, and there is no min-content-height, then we
need some small non-zero value to get started. The idea is to always
ask for at least enough to fit the horizontal scrollbar.

Simply put, this should be the mirror image of the corresponding width
calculation code.

Those who got used to the buggy behaviour might notice that their
GtkScrolledWindows are not as tall as they used to be.

Fall out from 55196a705f00564a44647bfc97981db0a783369a

https://bugzilla.gnome.org/show_bug.cgi?id=766530

gtk/gtkscrolledwindow.c

index 2f2b802f02e0323381c99e09d8bf87289df6fcd7..884e37747b7945bfe198ce7bd5a47c8019c2d995 100644 (file)
@@ -1787,10 +1787,10 @@ gtk_scrolled_window_measure (GtkCssGadget   *gadget,
                  natural_req.height = MAX (natural_req.height, priv->min_content_height);
                  extra_height = -1;
                }
-             else if (policy_may_be_visible (priv->vscrollbar_policy) && !priv->use_indicators)
+             else if (policy_may_be_visible (priv->hscrollbar_policy) && !priv->use_indicators)
                {
-                 minimum_req.height += vscrollbar_requisition.height;
-                 natural_req.height += vscrollbar_requisition.height;
+                 minimum_req.height += hscrollbar_requisition.height;
+                 natural_req.height += hscrollbar_requisition.height;
                }
            }
        }